Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add tag categories for VM migration #16402

Merged
merged 1 commit into from
Nov 13, 2017

Conversation

smelamud
Copy link
Contributor

@smelamud smelamud commented Nov 6, 2017

Added special tag categories for purpose of mapping VMs, networks and storage for mass migration between providers using V2V.

@smelamud
Copy link
Contributor Author

smelamud commented Nov 6, 2017

@miq-bot assign martinpovolny

@martinpovolny
Copy link
Member

I have never touched this area. @tinaafitz, @chessbyte : can you, please, reassign?

@@ -943,3 +943,41 @@
:parent_id: 0
:default: true
:single_value: "1"
- :description: Transform VM
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This tag category will be than used also for network and storage mapping, we should use a bit more generic term like:
"Transformation".

@smelamud smelamud changed the title Add tag category 'Transform VM' Add tag categories for VM migration Nov 8, 2017
@miq-bot
Copy link
Member

miq-bot commented Nov 8, 2017

Checked commits smelamud/manageiq@5717f35~...2287113 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 🍪

@jelkosz
Copy link
Contributor

jelkosz commented Nov 8, 2017

LGTM.
@mgoldboi FYI

@jelkosz
Copy link
Contributor

jelkosz commented Nov 10, 2017

@miq-bot assign tinaafitz

@miq-bot miq-bot assigned tinaafitz and unassigned martinpovolny Nov 10, 2017
@tinaafitz
Copy link
Member

@smelamud Looks good.

@jelkosz
Copy link
Contributor

jelkosz commented Nov 10, 2017

@miq-bot assign martinpovolny

@miq-bot miq-bot assigned martinpovolny and unassigned tinaafitz Nov 10, 2017
Added special tag categories for purpose of mapping VMs, networks and
storage for mass migration between providers using V2V.
@smelamud
Copy link
Contributor Author

Squashed commits.

@martinpovolny martinpovolny merged commit 9f42e1b into ManageIQ:master Nov 13, 2017
@martinpovolny
Copy link
Member

Gaprindashvili?

@martinpovolny martinpovolny added this to the Sprint 73 Ending Nov 13, 2017 milestone Nov 13, 2017
@smelamud
Copy link
Contributor Author

@miq-bot add_labels gaprindashvili/yes

simaishi pushed a commit that referenced this pull request Nov 14, 2017
Add tag categories for VM migration
(cherry picked from commit 9f42e1b)
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 9d62c7a96a716f3da2c7b4accaeb44bb0bc0f972
Author: Martin Povolny <mpovolny@redhat.com>
Date:   Mon Nov 13 13:18:38 2017 +0100

    Merge pull request #16402 from smelamud/vm-transform-tags
    
    Add tag categories for VM migration
    (cherry picked from commit 9f42e1b1181eb89cad003e5b56de7e817fdff67a)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants